fix leapp docs#5512
Conversation
Greptile SummaryThis PR corrects the
Confidence Score: 5/5Safe to merge — a one-line documentation-only fix with no runtime impact. The change updates three line-range values in an RST emphasize-lines directive. Cross-checking against the actual Python file confirms the new ranges correctly land on the LEAPP import (line 20), the output-annotation block (lines 75-77), and the input-annotation block (lines 92-105). No logic, configuration, or API surface is touched. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "fix docs highlight misalignment" | Re-trigger Greptile |
There was a problem hiding this comment.
Review: PR #5512 — fix docs highlight misalignment
LGTM! Clean docs fix.
Changes reviewed:
- Corrects the install command to use
./isaaclab.sh -p -m pip install leapp(proper Isaac Lab Python env) instead of barepip install leapp. - Expands the tutorial intro with clearer setup steps (install LEAPP, copy tutorial file into task package).
- Fixes the
:emphasize-lines:directive from20, 100-118, 85-88→20, 75-77, 92-105to match actual source line numbers — this was the highlight misalignment. - Minor wording fix in the warning block.
No issues found. Ship it. 🚀
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Screenshots
Please attach before and after screenshots of the change if applicable.
Checklist
pre-commitchecks with./isaaclab.sh --formatconfig/extension.tomlfileCONTRIBUTORS.mdor my name already exists there